Skip to content

fix: Add posix in pre-deployment hook in azure.yaml#659

Closed
chaudhariniraj wants to merge 3 commits into
microsoft:devfrom
chaudhariniraj:dev
Closed

fix: Add posix in pre-deployment hook in azure.yaml#659
chaudhariniraj wants to merge 3 commits into
microsoft:devfrom
chaudhariniraj:dev

Conversation

@chaudhariniraj

Copy link
Copy Markdown
Contributor

Purpose

Add posix in pre-deployment hook in azure.yaml

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a POSIX (sh) variant of the existing preprovision hook messaging in azure.yaml so non-Windows environments see the same deployment guidance during azd execution.

Changes:

  • Added a hooks.preprovision.posix hook that prints deployment troubleshooting guidance.
  • Configured the POSIX hook to run via sh and be interactive, mirroring the existing Windows hook behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread azure.yaml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 17, 2026 08:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread azure.yaml Outdated
interactive: true
posix:
run: |
printf '\n\033[33m ---------------------------------------------------------------------------------------------------\033[0m\n\033[33m Note: If deployment fails or you encounter an issue, please open an issue with the deployment logs.\033[0m\n\033[36m https://github.com/microsoft/content-processing-solution-accelerator/issues\033[0m\n\033[33m ---------------------------------------------------------------------------------------------------\033[0m\n\n'
Copilot AI review requested due to automatic review settings July 17, 2026 08:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread azure.yaml

metadata:
template: content-processing@1.0
name: content-processinge@1.0
Comment thread azure.yaml
Write-Host ""
shell: pwsh
interactive: true
# # yaml-language-server: $schema=https://raw.githubusercontent.com/Azure/azure-dev/main/schemas/v1.0/azure.yaml.json
@chaudhariniraj chaudhariniraj closed this by deleting the head repository Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants